Skip to content

Testing the last update of the Activitymodel#9

Open
mfbenitezp wants to merge 2 commits intoanetobradley:activitymodelfrom
mfbenitezp:activitymodel
Open

Testing the last update of the Activitymodel#9
mfbenitezp wants to merge 2 commits intoanetobradley:activitymodelfrom
mfbenitezp:activitymodel

Conversation

@mfbenitezp
Copy link
Copy Markdown
Collaborator

@mfbenitezp mfbenitezp commented Apr 19, 2022

@patricia-ternes I have tested the last update. Here are my results and comments. I have also added into the review.

  1. I think the README can also includes some instructions on how o run the model. More instructions in how to fin the right readme to make it run.
  2. In the readme of /config, is there any way to run it to the whole local authorities?
  3. Once the required parameters are set, the rest of the text can be part of some section like extended or expand the model to the users needs.
  4. I replaced the hi print for another message.
  5. Here are my images results for Leeds.

image
image

A question I have based on the results for Leeds is how do we know when the PS is not enough and there are some limitations on the pipeline, could be based on the EPC data, data sources or the required granularity. I guess that might included in the paper you've mentioned. But it would be good to have some kind of threshold to describe when the results should be consider with caution.

@mfbenitezp mfbenitezp changed the title Testig the last update of the Activitymodel Testing the last update of the Activitymodel Apr 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant